Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test articulations against vehicle dynamics #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

michalpelka
Copy link
Collaborator

No description provided.

Copy link
Contributor

@jhanca-robotecai jhanca-robotecai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test is correct, but please fix typos in the readme

- Check if messages are received and processed
- Check Vehicle dynamic

# Test Perquisite
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Test Perquisite
# Test Prerequisites


## Step 3

Press `Ctrl+G` to enter game mode.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Press `Ctrl+G` to enter game mode.
Press `Ctrl+G` to enter the game mode.

/tf_static [tf2_msgs/msg/TFMessage]
```

Detailed list of topics may by different, but:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Detailed list of topics may by different, but:
The detailed list of topics may be different, but:


- Check if ROS2 Control creates proper topics
- Check if messages are received and processed
- Check Vehicle dynamic
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Check Vehicle dynamic
- Check Vehicle dynamics


Leave the terminal open - it will be needed in the next steps.

> Note: make sure, no ROS services are runing on your computer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
> Note: make sure, no ROS services are runing on your computer
> Note: make sure, no ROS services are running on your computer

In the terminal run:

```bash
ros2 topic list
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ros2 topic list
ros2 topic list -t

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants